Skip to content

[3.7] bpo-35726: Add test for QueueHandler with multiple handlers (GH-11659)#12714

Closed
miss-islington wants to merge 1 commit into
python:3.7from
miss-islington:backport-2dad960-3.7
Closed

[3.7] bpo-35726: Add test for QueueHandler with multiple handlers (GH-11659)#12714
miss-islington wants to merge 1 commit into
python:3.7from
miss-islington:backport-2dad960-3.7

Conversation

@miss-islington

@miss-islington miss-islington commented Apr 7, 2019

Copy link
Copy Markdown
Contributor

(cherry picked from commit 2dad960)

Co-authored-by: Xtreak tir.karthi@gmail.com

https://bugs.python.org/issue35726

…-11659)

(cherry picked from commit 2dad960)

Co-authored-by: Xtreak <tir.karthi@gmail.com>
@tirkarthi

Copy link
Copy Markdown
Member

@vsajip #11537 needs to be backported to 3.7 so that this test doesn't fail for 3.7 https://travis-ci.org/python/cpython/jobs/516782296#L2065.

@miss-islington

Copy link
Copy Markdown
Contributor Author

@tirkarthi and @vsajip: Status check is done, and it's a failure ❌ .

2 similar comments
@miss-islington

Copy link
Copy Markdown
Contributor Author

@tirkarthi and @vsajip: Status check is done, and it's a failure ❌ .

@miss-islington

Copy link
Copy Markdown
Contributor Author

@tirkarthi and @vsajip: Status check is done, and it's a failure ❌ .

@miss-islington

Copy link
Copy Markdown
Contributor Author

@tirkarthi and @vsajip: Status check is done, and it's a failure ❌ .

@ned-deily ned-deily closed this May 2, 2019
@miss-islington miss-islington deleted the backport-2dad960-3.7 branch May 2, 2019 17:03
@tirkarthi

Copy link
Copy Markdown
Member

@ned-deily This adds test for #12716 . The reason for failure being #12716 has to be merged first for this to pass. Now that #12716 is merged it helps in backporting the test too to avoid regressions.

@ned-deily

Copy link
Copy Markdown
Member

miss-islington was too quick. Created another backport PR #13061.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

awaiting review skip news tests Tests in the Lib/test dir

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants